-
Notifications
You must be signed in to change notification settings - Fork 256
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Update Dockerfile for get-params to use EL9 #2147
Update Dockerfile for get-params to use EL9 #2147
Conversation
scripts/installer/Dockerfile
Outdated
|
||
RUN dnf -y install https://yum.puppetlabs.com/puppet7/puppet7-release-el-8.noarch.rpm | ||
RUN dnf -y install https://yum.puppetlabs.com/puppet7/el/9/x86_64/puppet7-release-7.0.0-19.el9.noarch.rpm |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I couldn't find a puppet7-release-el-9.noarch.rpm
so I just used this one. Probably not best to hard code a specific version like this.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
RUN dnf -y install https://yum.puppetlabs.com/puppet7/el/9/x86_64/puppet7-release-7.0.0-19.el9.noarch.rpm | |
RUN dnf -y install https://yum.puppet.com/puppet7-release-el-9.noarch.rpm |
scripts/installer/Dockerfile
Outdated
|
||
RUN dnf -y install https://yum.puppetlabs.com/puppet7/puppet7-release-el-8.noarch.rpm | ||
RUN dnf -y install https://yum.puppetlabs.com/puppet7/el/9/x86_64/puppet7-release-7.0.0-19.el9.noarch.rpm |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
RUN dnf -y install https://yum.puppetlabs.com/puppet7/el/9/x86_64/puppet7-release-7.0.0-19.el9.noarch.rpm | |
RUN dnf -y install https://yum.puppetlabs.com/puppet7-release-el-9.noarch.rpm |
at some point they fucked up the puppetX
folders and never updated the release symlinks in there 🤷♀️
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
awesome
26af5f4
to
2c972e0
Compare
CentOS stream 8 is no longer available so we should update this to EL9.